Skip to content

Added AASTeX (PSF), removed title from AASTeX#276

Merged
kelockhart merged 2 commits intoadsabs:masterfrom
kelockhart:aastex-title
May 23, 2025
Merged

Added AASTeX (PSF), removed title from AASTeX#276
kelockhart merged 2 commits intoadsabs:masterfrom
kelockhart:aastex-title

Conversation

@kelockhart
Copy link
Member

Per further discussions w/ AAS:

  • most AASTeX should not include the record title
  • AASTeX for PSJ should include the record title

Rolled back the previous addition of the title for /aastex, made a new endpoint, /aastex-psj, that retains the addition of the title.

@coveralls
Copy link

coveralls commented May 20, 2025

Coverage Status

coverage: 97.09% (-0.4%) from 97.475%
when pulling 90eb0f0 on kelockhart:aastex-title
into c466d9a on adsabs:master.

@kelockhart kelockhart requested a review from tjacovich May 22, 2025 16:59
Copy link

@tjacovich tjacovich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I would also just add the endpoint /aastex-psj in the README.

@kelockhart kelockhart merged commit 64bdb48 into adsabs:master May 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants